Skip to content

Remove legacy alias skills (create-pr, skill-creator)#110

Merged
dcramer merged 4 commits intomainfrom
remove-create-pr-skill
Apr 7, 2026
Merged

Remove legacy alias skills (create-pr, skill-creator)#110
dcramer merged 4 commits intomainfrom
remove-create-pr-skill

Conversation

@sentry-junior
Copy link
Copy Markdown
Contributor

@sentry-junior sentry-junior Bot commented Apr 7, 2026

Clean break from legacy alias skills. No compat shims, no forwarding aliases — just canonical skill names.

Removed

create-pr

Alias for pr-writer. Was actively causing routing problems — agents were landing on create-pr instead of the canonical skill.

skill-creator

Alias for skill-writer. Same problem: alias existence invites routing to the wrong skill.

Also cleaned up

  • .claude/settings.json — removed both from the allow list
  • AGENTS.md — removed alias mentions
  • claude-settings-audit/SKILL.md — removed expected entries for both
  • skill-writer/SOURCES.md and skill-writer/references/synthesis-path.md — removed skill-creator source references

junior added 2 commits April 7, 2026 16:54
- .claude/settings.json: drop Skill(sentry-skills:create-pr) from allow list
- AGENTS.md: remove create-pr from alias skill example
- claude-settings-audit/SKILL.md: drop create-pr from expected allow list
skill-creator was a compatibility alias for skill-writer. removing it
entirely as part of a clean break from legacy alias skills. no aliases,
no compat shims — just the canonical skill names.

removes:
- plugins/sentry-skills/skills/skill-creator/SKILL.md
- .claude/settings.json allow entry
- AGENTS.md alias mention
- claude-settings-audit/SKILL.md expected entry
- skill-writer/SOURCES.md source table row
- skill-writer/references/synthesis-path.md references
@sentry-junior sentry-junior Bot changed the title remove create-pr skill Remove legacy alias skills (create-pr, skill-creator) Apr 7, 2026
…le alias guidance

Co-Authored-By: junior <noreply@sentry.io>
@dcramer dcramer merged commit e69b219 into main Apr 7, 2026
12 checks passed
@dcramer dcramer deleted the remove-create-pr-skill branch April 7, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants